-
Notifications
You must be signed in to change notification settings - Fork 96
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed issue #254 - auto "fireuping" sup-config, if there are no any configured sources #358
base: develop
Are you sure you want to change the base?
Conversation
test how it works with:
|
Ok, @eMBee . I will make these tests. What should I do after that? Is there anything wrong? |
report on how each test went and, see if the script was started in every case. if the tests pass, then there is nothing to do, otherwise, look into what needs to be fixed to make that test work. |
Thank you for your answer. I will finish tests ASAP and give results there |
First test report: script started without any problem |
Second test: mistakes found. Runtime error. Improving |
Now, after my changes, all tests work |
make sure recovering from ncurses setup work. starting and stopping On Tue, Dec 30, 2014 at 11:16 AM, Tymon Radzik [email protected]
|
@@ -152,10 +152,18 @@ end | |||
module_function :start_cursing, :stop_cursing | |||
|
|||
Index.init | |||
Index.lock_interactively or exit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are you doing this? Make sure you understand all the implications of the changes you are doing as well as testing it. Changing this line is not something you do without careful thought and a good overview of how sup works.
I have fixed that as in disscussion. Now everything should be ok. Please also take a look on #359 |
this works, but i wonder if we need do consider more cases. right now, if i go through config, but explicitly do not add sources, it will ask me again every time until i have at least one source. i think that once config has been called, it should not be called again unless explicity invoked. |
as mentioned before: in commit messages: don't write "Added eMBee's suggestion" but explain what the actual change/suggestion is |
to sup devs: is a file to mark configured state ok? or is there a better way to mark that? some variable that can be saved? |
What about just checking if it matches default config and whether there are any sources, then ask the user whether he wants to launch sup-config (I can't think of any use case where the user would want to work without any sources, any previously indexed messages would be useless since they rely on the source to be able to open them). If there is, then there could be a config option that says something like 'do-not-ask-to-start-sup-config-again'. |
@@ -156,6 +156,16 @@ Index.lock_interactively or exit | |||
|
|||
begin | |||
Redwood::start | |||
Redwood::SourceManager.load_sources |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this not done twice now (in case of configured)?
Added automatic fire-uping "sup-config" if there are no any configured sources.